-
Notifications
You must be signed in to change notification settings - Fork 102
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix/synthese test #2950
Fix/synthese test #2950
Conversation
0a6ab08
to
240f8a0
Compare
5a3ec28
to
20fd21f
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #2950 +/- ##
===========================================
+ Coverage 78.44% 78.45% +0.01%
===========================================
Files 89 89
Lines 7199 7204 +5
===========================================
+ Hits 5647 5652 +5
Misses 1552 1552
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
bdd939f
to
7bda3ce
Compare
dc18deb
to
693f911
Compare
r = self.client.get(url) | ||
assert r.status_code == Unauthorized.code | ||
|
||
set_logged_user(self.client, users["self_user"]) | ||
|
||
r = self.client.get(url) | ||
assert r.status_code == 200 | ||
validate_json(instance=r.json, schema=schema) | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
doublons ?
) | ||
|
||
|
||
class NestedObs(Schema): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
docstring
693f911
to
f235b99
Compare
Utilisation de marshmallow pour la validation du retour de la route get_observations_for_web
Contrôle de la présence des champs obligatoires